Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

drivers/ft5x06: introduce conversion for X and Y coordinates #19867

Merged
merged 3 commits into from
Aug 27, 2023

Conversation

gschorcht
Copy link
Contributor

@gschorcht gschorcht commented Aug 9, 2023

Contribution description

This PR provides the parameter option to define how the X and Y coordinates have to be converted.

To get coordinates from the touch panel that correspond to the display coordinates, it is often necessary to convert the coordinates from the touch pannel by swapping and mirroring them. For the sake of simplicity, possible rotations are additionally defined.

The PR includes PRs #19860 and #19866 to be compilable.

Testing procedure

tests/pkg/lvgl_touch should still work for the stm32f746g-disco board.

BOARD=stm32f746g-disco make -C tests/pkg/lvgl_touch

Issues/PRs references

Depends on PR #19860
Depends on PR #19866

@github-actions github-actions bot added Area: drivers Area: Device drivers Area: tools Area: Supplementary tools Area: boards Area: Board ports labels Aug 9, 2023
@gschorcht gschorcht added Type: new feature The issue requests / The PR implemements a new feature for RIOT State: waiting for other PR State: The PR requires another PR to be merged first CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 9, 2023
@riot-ci
Copy link

riot-ci commented Aug 9, 2023

Murdock results

✔️ PASSED

26ed7e6 boards/stm32f723e-disco: add touch panel conversion config

Success Failures Total Runtime
7937 0 7937 15m:23s

Artifacts

@gschorcht gschorcht added the State: WIP State: The PR is still work-in-progress and its code is not in its final presentable form yet label Aug 14, 2023
@gschorcht gschorcht force-pushed the drivers/ft5x06_xy_conversion branch from c66e7ed to 2844f5a Compare August 14, 2023 13:49
@gschorcht gschorcht removed the State: WIP State: The PR is still work-in-progress and its code is not in its final presentable form yet label Aug 14, 2023
@gschorcht gschorcht force-pushed the drivers/ft5x06_xy_conversion branch from 2844f5a to 622d959 Compare August 23, 2023 21:39
@aabadie aabadie removed the State: waiting for other PR State: The PR requires another PR to be merged first label Aug 25, 2023
Copy link
Contributor

@aabadie aabadie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

bors merge

@bors
Copy link
Contributor

bors bot commented Aug 25, 2023

👎 Rejected by PR status

@aabadie
Copy link
Contributor

aabadie commented Aug 25, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Aug 25, 2023

👎 Rejected by PR status

@benpicco benpicco added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 25, 2023
@aabadie
Copy link
Contributor

aabadie commented Aug 25, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Aug 25, 2023

👎 Rejected by PR status

@gschorcht
Copy link
Contributor Author

Why the hell doesn't bors like this PR? If I'm right, we had the same problem with release.notes PR and the only thing that helped was rebasing it. I will rebase the PR and try again.

To obtain coordinates from the touch panel that correspond to the display coordinates, it is often necessary to convert the coordinates from the touch display by swapping and mirroring. For the sake of simplicity, possible rotations are additionally defined.
@gschorcht gschorcht force-pushed the drivers/ft5x06_xy_conversion branch from 622d959 to 26ed7e6 Compare August 26, 2023 15:49
@aabadie aabadie added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Aug 27, 2023
@aabadie
Copy link
Contributor

aabadie commented Aug 27, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Aug 27, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 9a5675b into RIOT-OS:master Aug 27, 2023
@gschorcht
Copy link
Contributor Author

@aabadie Thanks for reviewing and merging.

@gschorcht gschorcht deleted the drivers/ft5x06_xy_conversion branch August 29, 2023 15:42
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.10 milestone Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: boards Area: Board ports Area: drivers Area: Device drivers Area: tools Area: Supplementary tools CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: new feature The issue requests / The PR implemements a new feature for RIOT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants